-
-
Notifications
You must be signed in to change notification settings - Fork 252
Adapt some pages to ReScript 11 including records and variants #730
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great.
Perhaps @zth would also like to take a look.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one small comment that I think should be changed now. Outside of that I think we can merge this and just continue working on whatever else was commented on.
Also, another task - we need a section on spreading variants.
@zth I assigned you to some new issues. |
@fhammerschmidt tests seem to fail, but otherwise this should be good to go, right? |
I have another branch open to fix the tests. Should be no problem to merge for now since it is not the main branch anyway. |
* Adapt some pages to ReScript 11 including records and variants * Some better wordings * Variants: Add some TODOs
* Adapt some pages to ReScript 11 including records and variants * Some better wordings * Variants: Add some TODOs
…ipt-lang#730) * Adapt some pages to ReScript 11 including records and variants * Some better wordings * Variants: Add some TODOs
No description provided.